Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misc: allow HiDPI Screen running wayland to use cypress window/browser #30109

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

AtofStryker
Copy link
Contributor

Additional details

A revisit of #29366. When merging in #29366, there were issues detecting the --ozone-platform=auto in older versions of linux distributions, the --ozone-platform-hint argument is in place to not make this a critical error when testing the binary

From original PR #29366:

  1. High demand of HiDPI Screen and multi monitor setups make people tend to use fractional scaling for each their own monitor differently. Problem happens when using chromium based app that can lead to blurriness of the UI. By providing simple flag to the electron arguments --ozone-platform=auto which literally can backwards compatible will help Wayland users to use chromium/electron without blurry texts, the flag also compatible with x11 window system if it's the only available option.

  2. Just small change to add the flag to the spawn.js file

Steps to test

It's already tested here

How has the user experience changed?

PR Tasks

@AtofStryker AtofStryker self-assigned this Aug 26, 2024
Copy link

cypress bot commented Aug 26, 2024

cypress    Run #56807

Run Properties:  status check passed Passed #56807  •  git commit f0a7e2adfd: misc: allow HiDPI Screen running wayland to use cypress window/browser [run ci]
Project cypress
Branch Review chore/support_hidpi_wayland
Run status status check passed Passed #56807
Run duration 48m 17s
Commit git commit f0a7e2adfd: misc: allow HiDPI Screen running wayland to use cypress window/browser [run ci]
Committer AtofStryker
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 116
View all changes introduced in this branch ↗︎
UI Coverage  null%
  Untested elements 0  
  Tested elements 0  
Accessibility  null%
  Failed rules  0 critical   0 serious   0 moderate   0 minor
  Failed elements 0  

@AtofStryker AtofStryker marked this pull request as ready for review August 26, 2024 18:50
@AtofStryker AtofStryker merged commit 50335a0 into develop Aug 26, 2024
130 of 132 checks passed
@AtofStryker AtofStryker deleted the chore/support_hidpi_wayland branch August 26, 2024 19:50
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Aug 27, 2024

Released in 13.14.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.14.0, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Aug 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wayland support on Linux
3 participants